Skip to content

[sonic-yang-models]: Add TC_ACTION to sonic-acl#28193

Open
AnantKishorSharma wants to merge 1 commit into
sonic-net:masterfrom
AnantKishorSharma:acl-set-tc-yang
Open

[sonic-yang-models]: Add TC_ACTION to sonic-acl#28193
AnantKishorSharma wants to merge 1 commit into
sonic-net:masterfrom
AnantKishorSharma:acl-set-tc-yang

Conversation

@AnantKishorSharma

@AnantKishorSharma AnantKishorSharma commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

Why I did it

Model the new TC_ACTION ACL rule action (set traffic class) in the ACL YANG model so CONFIG_DB validation accepts it. Companion to the orchagent change that maps TC_ACTION to SAI_ACL_ENTRY_ATTR_ACTION_SET_TC.

How I did it

  • Add leaf TC_ACTION { type uint8; } to sonic-acl.yang.j2.
  • Add positive (0/7/255) and negative (out-of-range 256) YANG model tests.

How to verify it

  • pytest on sonic-yang-models (yang_model_tests) passes; a valid TC_ACTION renders and an out-of-range value is rejected (Bounds).

Related PRs:

Signed-off-by: Anant Kishor Sharma <anant.kishor-sharma@hpe.com>
@mssonicbld

Copy link
Copy Markdown
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@AnantKishorSharma

Copy link
Copy Markdown
Contributor Author

/azpw run

@mssonicbld

Copy link
Copy Markdown
Collaborator

⚠️ Notice: /azpw run only runs failed jobs now. If you want to trigger a whole pipline run, please rebase your branch or close and reopen the PR.
💡 Tip: You can also use /azpw retry to retry failed jobs directly.

Retrying failed(or canceled) jobs...

@mssonicbld

Copy link
Copy Markdown
Collaborator

Build not found. Please close and reopen the PR or rebase your branch to trigger a new build.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants